fix(deps): patch nitropack so comark plugins stay external on Nuxt 4.6 - #59

Correctness · e2e/routes.test.ts:40 · in Route Crawl ＆ SSR Tests

it.each(pages) skips both image checks when a docs page has no og:image, so the test passes even though that page calls defineOgImage.

Fix: Assert that docs pages have an og:image before checking the image response.

Component view — E2E Testing Suite

Internal runner setup, bundle integrity checks, and route crawl assertions under Vitest.

• Run npx skills add coldteadotai/pr-lens, then tell your coding agent: "Diagram the change you just made with PR Lens and attach it to the pull request."
• Run npx @coldtea/pr-lens-cli analyze --base origin/main on a branch, then npx @coldtea/pr-lens-cli render .pr-lens/graph.json. Same lenses, your own model key, before the pull request exists
• Click the link under each diagram to open it on a canvas you can zoom, pan and step through
• The diagrams are links. Click one to open it on the canvas, then press W or click play to walk through the change
• Open a diagram on the canvas, then press W or click play to walk through the change one step at a time
• The CLI's render reads .github/pr-lens.yml and applies your renames, exclusions and lane pins at draw time
• Set github.comment.collapsed: true in .github/pr-lens.yml to fold the comment behind one View architecture and data flow row. Drawing still runs as before
• Add .github/workflows/pr-lens.yml with coldteadotai/pr-lens/packages/action@v0 and your model provider's key as its api-key to run PR Lens from your own CI. Any /chat/completions endpoint works
• After a push, tick Redraw to draw the latest commit
• Switch GitHub to dark mode and the diagrams follow. The moving dots are this pull request's data in motion

• X
• Bluesky
• Mastodon
• Reddit
• LinkedIn

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5c2db1a3-8074-4ad6-9216-2ad231a63e06

📥 Commits

Reviewing files that changed from the base of the PR and between efb48e0 and e0799d0.

📒 Files selected for processing: 1

• .github/workflows/ci.yml

🚧 Files skipped from review as they are similar to previous changes: 1

• .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

The change patches Nitro route generation and package-export resolution. It adds a Vitest end-to-end project that starts the built playground server and checks bundled imports and routes. CI runs the end-to-end tests after the build.

Changes: Build and end-to-end validation

Layer / File(s)	Summary
Nitro route and export resolution changes patches/nitropack@2.13.4.patch, pnpm-workspace.yaml	The Nitro patch emits header-only Vercel routes separately from redirects and adds matching for pattern exports during external resolution. The workspace applies the patch to nitropack@2.13.4.
Vitest end-to-end project and server setup vitest.config.ts, package.json, test/e2e/setup.ts	Vitest separates unit and end-to-end projects. End-to-end setup starts the built playground server, waits for /robots.txt, provides baseURL, and returns a cleanup function.
End-to-end checks and CI execution test/e2e/*, .github/workflows/ci.yml	The tests check bundled imports, sitemap pages, Open Graph images, agent routes, and markdown responses. CI runs the end-to-end project after the build with GitHub token and branch environment values.

Priority: ⬆️ High

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Bug fix

Sequence Diagram(s)

sequenceDiagram
  participant CI as CI workflow
  participant Vitest
  participant Setup as test/e2e/setup.ts
  participant Server as Playground server
  participant RouteTests as Route tests
  participant BundleTests as Bundle tests
  CI->>Vitest: Run pnpm test:e2e after build
  Vitest->>Setup: Run global setup
  Setup->>Server: Start built server and wait for /robots.txt
  Setup-->>Vitest: Provide baseURL
  Vitest->>RouteTests: Run route checks
  RouteTests->>Server: Request pages, images, and markdown
  Vitest->>BundleTests: Run bundle import checks
  BundleTests->>Server: Read built server output

Loading

The change patches Nitro externals handling and adds end-to-end checks in CI. No concrete merge-blocking risk is established from the supplied evidence.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 4 files. (1 skipped: 1…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly identifies the main change: patching NitroPack dependencies to keep Comark plugins external for Nuxt 4.6. It is concise and specific.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 4 files. (1 skipped: 1 unsupported.)

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• X
• Mastodon
• Reddit
• LinkedIn